Feat: Aplica PEP 8, añade hook pre-commit y modifica el proyecto#16
Feat: Aplica PEP 8, añade hook pre-commit y modifica el proyecto#16joselofierro wants to merge 3 commits intocamigomezdev:mainfrom
Conversation
codebreaker.py
Outdated
| elif number[idx] in NUMBER_TO_GUESS: | ||
| resultTrack += '_' | ||
| else: | ||
| resultTrack += '#' |
There was a problem hiding this comment.
Puedes modificar la logica para que agrupe primero las (X), luego los (_) bajos y por ultimo los (#)?
codebreaker.py
Outdated
| if number == NUMBER_TO_GUESS: | ||
| return True | ||
|
|
||
| resultTrack = '' |
There was a problem hiding this comment.
Recuerda que PEP8 recomienda utilizar snake_case en vez de camelCase para el nombramiento de las variables
codebreaker.py
Outdated
|
|
||
| return resultadoX+resultado_ | ||
|
|
||
| def adivinar(self, number: str = None) -> Union[bool, str]: |
There was a problem hiding this comment.
Puedes dar un nombre mas explicito a esta función?
codebreaker.py
Outdated
| else: | ||
| resultTrack += '#' | ||
|
|
||
| return resultTrack |
There was a problem hiding this comment.
Estas mezclando español e inglés en los nombres de tus variables, puedes unificarlo en un solo idioma?
main.py
Outdated
|
|
||
| intentos_totales = 10 | ||
| TOTAL_ATTEMPTS = 5 | ||
| ATTEMPTS = 1 |
There was a problem hiding this comment.
attepts funciona como una variable ya que en cada iteración cambia su valor, escribirlo en mayuscula indicaría que es una constante y que siempre su valor será 1
main.py
Outdated
| number = input('Adivina el número de 4 cifras: ') | ||
| resolve = codebreaker.adivinar(number) | ||
| print(resolve) | ||
| if resolve is True: |
There was a problem hiding this comment.
Puedes modificar este if para que solo pregunte if resolve:, si resolve puede ser otros tipos de datos que tambien son funcionan como verdadero trata de modificar el codigo para que siempre sea un valor booleano.
2 participants
Happy path
git clone https://github.com/joselofierro/dojo_pep8.gitcd dojo_pep8virtualenv -p /usr/bin/python3 venvsource venv/bin/activatepip install -r requirements.txtpython main.py